Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix arch check for PowerShell enumeration #114292

Merged
merged 2 commits into from Jan 13, 2021
Merged

Fix arch check for PowerShell enumeration #114292

merged 2 commits into from Jan 13, 2021

Conversation

TylerLeonhardt
Copy link
Member

This PR fixes #114248

We used the wrong check for architecture.

Additionally, on some AzDO instances PowerShell Core is not installed so we can't make that assuption anymore... so this also fixes the test to not make that assuption.

@TylerLeonhardt TylerLeonhardt changed the title Delay animating until content is prepared to prevent weird flying elements. Fix arch check for PowerShell enumeration Jan 13, 2021
@TylerLeonhardt TylerLeonhardt added this to the January 2021 milestone Jan 13, 2021
@TylerLeonhardt TylerLeonhardt merged commit 2d892ae into master Jan 13, 2021
@TylerLeonhardt TylerLeonhardt deleted the TylerLeonhardt/fix-win32-test-take2 branch January 13, 2021 17:53
@github-actions github-actions bot locked and limited conversation to collaborators Feb 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Tyriar Tyriar Tyriar approved these changes

Assignees

@TylerLeonhardt TylerLeonhardt

Labels
None yet
Projects
None yet
Milestone
January 2021
Development

Successfully merging this pull request may close these issues.

Can enumerate PowerShells test failure
2 participants
@TylerLeonhardt @Tyriar